-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[WIP] Design matrix and array of voxels in top 20% of t-statistics #34
Conversation
Dev: update linear_modeling.py
[WIP] Update design matrix and create array of indices of voxels with highest t-statistics
X[:,2] = pos | ||
X[:,3] = np.linspace(-1, 1, n_trs) | ||
return X | ||
def plot_design_matrix(X): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please include two new lines between function definitions for readability
if you do yapf -i code/utils/linear_modeling.py this is one of the styling that it'll do automatically
Thanks! I addressed most of those, except line 43. I will push up those changes and the tests tomorrow. I'm having some problems importing into the test_linear_modeling.py file |
Add slides, markdown, and necessary pics
I added the files needed for our slides, but don't know how to have those in a pull request by itself |
Added tests for parse_demographics
Fixed tests and some functions
Design matrix and array of voxels in top 20% of t-statistics
Still in the process of writing tests, please check to see if I need to redo the functions to make it more efficient